1. Nov 19, 2019
    • Francis Visoiu Mistrih's avatar
      [Remarks][Driver] Use different remark files when targeting multiple architectures · b4e2b112
      Francis Visoiu Mistrih authored
      When the driver is targeting multiple architectures at once, for things
      like Universal Mach-Os, we need to emit different remark files for each
      cc1 invocation to avoid overwriting the files from a different
      invocation.
      
      For example:
      
      $ clang -c -o foo.o -fsave-optimization-record -arch x86_64 -arch x86_64h
      
      will create two remark files:
      
      * foo-x86_64.opt.yaml
      * foo-x86_64h.opt.yaml
      b4e2b112
    • Yi Kong's avatar
      1689ad27
    • Vedant Kumar's avatar
      [Signal] Allow llvm clients to opt into one-shot SIGPIPE handling · 4624e83c
      Vedant Kumar authored
      Allow clients of the llvm library to opt-in to one-shot SIGPIPE
      handling, instead of forcing them to undo llvm's SIGPIPE handler
      registration (which is brittle).
      
      The current behavior is preserved for all llvm-derived tools (except
      lldb) by means of a default-`true` flag in the InitLLVM constructor.
      
      This prevents "IO error" crashes in long-lived processes (lldb is the
      motivating example) which both a) load llvm as a dynamic library and b)
      *really* need to ignore SIGPIPE.
      
      As llvm signal handlers can be installed when calling into libclang
      (say, via RemoveFileOnSignal), thereby overriding a previous SIG_IGN for
      SIGPIPE, there is no clean way to opt-out of "exit-on-SIGPIPE" in the
      current model.
      
      Differential Revision: https://reviews.llvm.org/D70277
      4624e83c
    • Jon Chesterfield's avatar
      [libomptarget][nfc] Move some source into common from nvptx · 5a4a05d7
      Jon Chesterfield authored
      Summary:
      [libomptarget][nfc] Move some source into common from nvptx
      
      Moves some source that compiles cleanly under amdgcn into a common subdirectory
      Includes some non-trivial files and some headers. Keeps the cuda file extension.
      
      The build systems for different architectures seem unlikely to have much in
      common. The idea is therefore to set include paths such that files under
      common/src compile as if they were under arch/src as the mechanism for sharing.
      In particular, files under common/src need to be able to include target_impl.h.
      
      The corresponding -Icommon is left out in favour of explicit includes on the
      basis that the it makes it clearer which files under common are used by a given
      architecture.
      
      Reviewers: jdoerfert, ABataev, grokos
      
      Reviewed By: ABataev
      
      Subscribers: jfb, mgorny, openmp-commits
      
      Tags: #openmp
      
      Differential Revision: https://reviews.llvm.org/D70328
      5a4a05d7
    • Michael Liao's avatar
      Fix shared lib build. · 17e37ba5
      Michael Liao authored
      17e37ba5
    • Adrian Prantl's avatar
      Replace bitfield in lldb::Type with byte-sized members. (NFC) · d4f18f11
      Adrian Prantl authored
      Due to alginment and packing using separate members takes up the same
      amount of space, but makes it far less cumbersome to deal with it in
      constructors etc.
      d4f18f11
    • Julian Lettner's avatar
      7e4d386b
    • Julian Lettner's avatar
      Bump sanitizer iOS deployment target to 9.0 · 4d4066a5
      Julian Lettner authored
      Bump default value for `-miphoneos-version-min=x` to 9.0 (from 8.0).
      TSan tests that use thread local storage `__thread` (e.g., tls_race.cpp)
      are failing to link for the iOS simulator (arch=x86_64 and
      -miphoneos-version-min=8.0) on internal/master (will be submitted to
      Glacier train):
      ```
      ld: targeted OS version does not support use of thread local variables in _main for architecture x86_64
      ```
      
      iOS 9.0 was released 2015.
      4d4066a5
    • Julian Lettner's avatar
      Bump sanitizer macOS deployment target to 10.10 · b87fc09d
      Julian Lettner authored
      Bump default value for `SANITIZER_MIN_OSX_VERSION` to 10.10 (from 10.9).
      TSan does not work on macOS 10.9 and a nice error message is preferable
      to an "unreferenced symbol" error when loading the TSan runtime.
      
      We could try to only bump the deployment target for TSan, but we would
      have to invest into adding support for this to our CMake build and it
      does not seem worth it.  macOS 10.10 was released in 2014.
      
      rdar://31335781
      b87fc09d
    • Jonas Devlieghere's avatar
      [Docs] Add Python caveats under the development section · 0aed6486
      Jonas Devlieghere authored
      This adds a page named Caveats with a section on some of the things to
      be aware of related to Python. It's a question we've seen more than once
      pop up and I think it's good to have it documentation on the website.
      Even though some of it might be useful to users, I still put it under
      "development" because it requires some understanding of how LLDB is
      built.
      
      Differential revision: https://reviews.llvm.org/D70252
      0aed6486
    • Fangrui Song's avatar
      [yaml2obj][test] Move tests to binary format specific subdirectories · 63f0f54c
      Fangrui Song authored
      Create COFF/, ELF/, and Minidump and move tests there.
      
      Also
      
      * Rename `*.test` to `*.yaml`
      * For yaml2obj RUN lines, use `-o %t` instead of `> %t` for consistency.
        We still have tests that check stdout is the default output, e.g.
        multi-doc.test
      * Update tests to consistently use `##` for comments.
        `#` is for RUN and CHECK lines.
      * Merge symboless-relocation.yaml and invalid-symboless-relocation.yaml to ELF/relocation-implicit-symbol-index.test
      
      Reviewed By: grimar, jhenderson
      
      Differential Revision: https://reviews.llvm.org/D70264
      63f0f54c
    • Joel E. Denny's avatar
      [Attr] Fix `-ast-print` for `asm` attribute · c85fa79d
      Joel E. Denny authored
      Without this fix, the tests introduced here produce the following
      assert fail:
      
      ```
      clang: /home/jdenny/llvm/clang/include/clang/Basic/AttributeCommonInfo.h:163: unsigned int clang::AttributeCommonInfo::getAttributeSpellingListIndex() const: Assertion `(isAttributeSpellingListCalculated() || AttrName) && "Spelling cannot be found"' failed.
      ```
      
      The bug was introduced by D67368, which caused `AsmLabelAttr`'s
      spelling index to be set to `SpellingNotCalculated`.
      
      Reviewed By: aaron.ballman
      
      Differential Revision: https://reviews.llvm.org/D70349
      c85fa79d
    • Francesco Petrogalli's avatar
      [dwarfgen] Fix initialization order error. [NFCI] · 015b2e69
      Francesco Petrogalli authored
      This commit fixes the `-Werror=reorder` builds.
      015b2e69
    • Alexey Bataev's avatar
      [OPENMP50]Fix PR44024: runtime assert in distribute construct. · c3eded06
      Alexey Bataev authored
      If the code is emitted for distribute construct, the nonmonotonic
      modifier should not be added.
      c3eded06
    • Julian Lettner's avatar
      [ASan] Mark test as UNSUPPORTED for iOS simulator · b0567be2
      Julian Lettner authored
      coverage-fork.cpp uses `fork()` which requires additional permissions
      in the iOS simulator sandbox.  We cannot use `sandbox-exec` to grant
      these permissions since this is a Posix (not Darwin) test.
      b0567be2
  2. Nov 18, 2019