[PATCH] D76620: [SYCL] Implement __builtin_unique_stable_name.

rjmccall added a comment. In D76620#2328031 , @erichkeane wrote: > In D76620#2328011 , @rjmccall wrote: > >> In D76620#2327988 , @erichkeane >> wrote: >> >>> In D76620#23279

[PATCH] D88154: Initial support for vectorization using Libmvec (GLIBC vector math library).

venkataramanan.kumar.llvm updated this revision to Diff 297925. venkataramanan.kumar.llvm added a comment. Updated the patch as per review comments received. The test cases are updated the checks based on the below comment from Francesco. ---Snip-- I think you are over-testing here. It is enough

[PATCH] D76620: [SYCL] Implement __builtin_unique_stable_name.

erichkeane added a comment. In D76620#2328305 , @rjmccall wrote: > In D76620#2328031 , @erichkeane > wrote: > >> In D76620#2328011 , @rjmccall wrote: >> >>> In D76620#232798

[PATCH] D85802: [clang] Add -fc++-abi= flag for specifying which C++ ABI to use

leonardchan marked an inline comment as done. leonardchan added inline comments. Comment at: clang/lib/Frontend/CompilerInvocation.cpp:3469 + + // The value can be empty, which indicates the system default should be used. + StringRef CXXABI = Args.getLastArgValue(OPT_fcxx_abi_E

[clang] 77bb3eb - Revert "[analyzer] NFC: Move path diagnostic consumer implementations to libAnalysis."

Author: Artem Dergachev Date: 2020-10-13T12:03:04-07:00 New Revision: 77bb3ebebbca13f0648beb433fbd1b06ba95a19c URL: https://github.com/llvm/llvm-project/commit/77bb3ebebbca13f0648beb433fbd1b06ba95a19c DIFF: https://github.com/llvm/llvm-project/commit/77bb3ebebbca13f0648beb433fbd1b06ba95a19c.dif

[clang] 662ed9e - Revert "[analyzer] NFC: Move IssueHash to libAnalysis."

Author: Artem Dergachev Date: 2020-10-13T12:07:28-07:00 New Revision: 662ed9e67adace3d011f42478bc8bcb1773a2821 URL: https://github.com/llvm/llvm-project/commit/662ed9e67adace3d011f42478bc8bcb1773a2821 DIFF: https://github.com/llvm/llvm-project/commit/662ed9e67adace3d011f42478bc8bcb1773a2821.dif

[clang] 10f1ca9 - Revert "[analyzer] NFC: Separate PathDiagnosticConsumer options from AnalyzerOptions."

Author: Artem Dergachev Date: 2020-10-13T12:07:36-07:00 New Revision: 10f1ca99b498347186ec74b01046ad292bde9a4c URL: https://github.com/llvm/llvm-project/commit/10f1ca99b498347186ec74b01046ad292bde9a4c DIFF: https://github.com/llvm/llvm-project/commit/10f1ca99b498347186ec74b01046ad292bde9a4c.dif

[PATCH] D89210: [Sema, CodeGen] Implement [[likely]] and [[unlikely]] in SwitchStmt

aaron.ballman added a comment. Thank you for the continued work on this feature! I'd like to better understand the behavior of fallthrough labels because I think there are two conceptual models we could follow. When a label falls through to another label, should both labels be treated as having

[PATCH] D78658: [clang][Frontend] Add missing error handling

dblaikie added a comment. In D78658#2327615 , @LemonBoy wrote: >> Perhaps this'd be more robust with ScopeExit? > > Not really, `OnError` is not executed when/if the function succeeds. Ah, sorry - ScopeExit has a 'release()' function intended to disengage

[clang] 0ec1cf1 - Revert "DirectoryWatcher: add an implementation for Windows"

Author: Reid Kleckner Date: 2020-10-13T12:35:22-07:00 New Revision: 0ec1cf13f2a4e31aa2c5ccc665c5fbdcd3a94577 URL: https://github.com/llvm/llvm-project/commit/0ec1cf13f2a4e31aa2c5ccc665c5fbdcd3a94577 DIFF: https://github.com/llvm/llvm-project/commit/0ec1cf13f2a4e31aa2c5ccc665c5fbdcd3a94577.diff

[PATCH] D84962: [PowerPC] Correct cpsgn's behaviour on PowerPC to match that of the ABI

Conanap updated this revision to Diff 297936. Conanap added a comment. Use regex match to check for variable order in IR instead. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84962/new/ https://reviews.llvm.org/D84962 Files: clang/lib/Headers/altivec.h clang/test/CodeGen/builtins-

[PATCH] D84962: [PowerPC] Correct cpsgn's behaviour on PowerPC to match that of the ABI

Conanap marked 3 inline comments as done. Conanap added a comment. Addressed Nemenja's concerns. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84962/new/ https://reviews.llvm.org/D84962 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D89184: Support complex target features combinations

echristo added a comment. Hi Peng, Looks interesting, but I think the language support needs some more comments about what's expected and in addition through everything else please? I've added some inline comments with places I think could use it for sure. -eric Comment at:

[PATCH] D89184: Support complex target features combinations

echristo requested changes to this revision. echristo added a comment. This revision now requires changes to proceed. Mark as requesting changes :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89184/new/ https://reviews.llvm.org/D89184 ___ c

[PATCH] D78699: [X86] Passing union type through register.

dyung added a comment. Hi, the test you added seems to pass both before and after your change, is this intended? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78699/new/ https://reviews.llvm.org/D78699

[clang] 4c10d65 - [AIX] Support two itanium alignment LIT testcases for AIX using regex

Author: Xiangling Liao Date: 2020-10-13T16:47:01-04:00 New Revision: 4c10d6508f546ee986df0994663984cb15354c08 URL: https://github.com/llvm/llvm-project/commit/4c10d6508f546ee986df0994663984cb15354c08 DIFF: https://github.com/llvm/llvm-project/commit/4c10d6508f546ee986df0994663984cb15354c08.diff

[PATCH] D89064: [AIX] Support two itanium alignment LIT testcases for AIX using regex

This revision was automatically updated to reflect the committed changes. Closed by commit rG4c10d6508f54: [AIX] Support two itanium alignment LIT testcases for AIX using regex (authored by Xiangling_L). Changed prior to commit: https://reviews.llvm.org/D89064?vs=297848&id=297950#toc Repositor

[PATCH] D87702: [Frontend] Add pragma align natural and sort out pragma pack stack effect

Xiangling_L marked 8 inline comments as done. Xiangling_L added inline comments. Comment at: clang/include/clang/Basic/LangOptions.def:340 +LANGOPT(AIXPragmaPack, 1, 0, "AIX #pragma pack handling") + jasonliu wrote: > Not sure if AIXPragmaPack is the best name

[PATCH] D89072: [CodeView] Emit static data members as S_CONSTANTs.

rnk added inline comments. Comment at: clang/test/CodeGenCXX/debug-info-static-member.cpp:144-147 +// For some reason, const_va is not emitted when the target is MS. +// NOT-MS: !DIDerivedType(tag: DW_TAG_member, name: "const_va", +// NOT-MS-SAME: line: [[@LINE-3]] +//

[PATCH] D78658: [clang][Frontend] Add missing error handling

LemonBoy updated this revision to Diff 297959. LemonBoy added a comment. Use `ScopeExit` instead of a bare lambda. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78658/new/ https://reviews.llvm.org/D78658 Files: clang/lib/Frontend/ASTUnit.cpp clang/unittests/Frontend/ASTUnitTest.cpp

[PATCH] D78699: [X86] Passing union type through register.

dyung added inline comments. Comment at: clang/test/CodeGen/X86/avx-union.c:2 +// RUN: %clang_cc1 -w -ffreestanding -triple x86_64-linux-gnu -target-feature +avx -emit-llvm -o %t %s || FileCheck < %t %s --check-prefix=CHECK, AVX +// RUN: %clang_cc1 -w -ffreestanding -triple x86_

[clang] e7fe3c6 - Basic: Simplify SourceManager::getBuffer overload, NFC

Author: Duncan P. N. Exon Smith Date: 2020-10-13T17:52:59-04:00 New Revision: e7fe3c6dfede8d5781bd000741c3dea7088307a4 URL: https://github.com/llvm/llvm-project/commit/e7fe3c6dfede8d5781bd000741c3dea7088307a4 DIFF: https://github.com/llvm/llvm-project/commit/e7fe3c6dfede8d5781bd000741c3dea70883

[PATCH] D87956: [WIP][IR] add fn attr for no_stack_protector; prevent inlining ssp into no-ssp

nickdesaulniers added inline comments. Comment at: llvm/include/llvm/Bitcode/LLVMBitCodes.h:610 + // TODO: reorder + ATTR_KIND_NO_STACK_PROTECT = 70, ATTR_KIND_STACK_PROTECT = 26, nickdesaulniers wrote: > any comments from reviewers before I go and do a tedi

[PATCH] D87956: [WIP][IR] add fn attr for no_stack_protector; prevent inlining ssp into no-ssp

nickdesaulniers updated this revision to Diff 297967. nickdesaulniers marked 2 inline comments as done. nickdesaulniers added a comment. - address todos and review feedback Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87956/new/ https://reviews.ll

[PATCH] D87956: [WIP][IR] add fn attr for no_stack_protector; prevent inlining ssp into no-ssp

nickdesaulniers updated this revision to Diff 297969. nickdesaulniers added a comment. - add -passes='cgscc(inline)' to inline_nossp.ll Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87956/new/ https://reviews.llvm.org/D87956 Files: clang/include

[PATCH] D87956: [WIP][IR] add fn attr for no_stack_protector; prevent inlining ssp into no-ssp

nickdesaulniers added inline comments. Comment at: llvm/docs/BitCodeFormat.rst:953 base 2 of the requested alignment, plus 1 +* bit 40: ``nossp`` ah, I should remove this; this block is legacy encodings which I do not modify. Comment at: l

[PATCH] D87956: [WIP][IR] add fn attr for no_stack_protector; prevent inlining ssp into no-ssp

nickdesaulniers updated this revision to Diff 297970. nickdesaulniers added a comment. - remove modification to docs about legacy fn attr encoding Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87956/new/ https://reviews.llvm.org/D87956 Files: cl

[PATCH] D89055: [analyzer] Wrong type cast occures during pointer dereferencing after type punning

ASDenysPetrov added a comment. @NoQ > Can we collapse this function further towards this goal? Say, why not pump > every region unconditionally through castRegion()? Does dispatchCast() use > castRegion() internally - and if it does, why are there so many branches in > this function? I've mad

[PATCH] D88345: [CUDA] Allow local `static const {__constant__, __device__}` variables.

jlebar accepted this revision. jlebar added inline comments. This revision is now accepted and ready to land. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:8163 "%select{__device__|__global__|__host__|__host__ __device__}0 functions">; -def err_cuda_nonglobal_

[PATCH] D89276: Support ObjC in IncludeInserter

compositeprimes updated this revision to Diff 297980. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89276/new/ https://reviews.llvm.org/D89276 Files: clang-tools-extra/unittests/clang-tidy/IncludeInserterTest.cpp Index: clang-tools-extra/unittes

[PATCH] D89348: clang/Basic: Replace ContentCache::getBuffer with Optional semantics

dexonsmith created this revision. dexonsmith added a reviewer: arphaman. Herald added subscribers: ributzka, martong. Herald added a reviewer: shafik. dexonsmith requested review of this revision. Remove `ContentCache::getBuffer`, which always returned a dereferenceable `MemoryBuffer*` and had a `

[PATCH] D89348: clang/Basic: Replace ContentCache::getBuffer with Optional semantics

dexonsmith added a comment. Note that this is split off from https://reviews.llvm.org/D66782. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89348/new/ https://reviews.llvm.org/D89348 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D57265: [PM/CC1] Add -f[no-]split-cold-code CC1 options to toggle splitting

compnerd updated this revision to Diff 297983. compnerd added a comment. Herald added subscribers: kerbowa, nhaehnle, jvesely. Herald added a project: clang. Rebase; there are a 4 more test failures to go through Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.

[clang] 8d03a97 - PR47805: Use a single object for a function parameter in the caller and

Author: Richard Smith Date: 2020-10-13T15:45:04-07:00 New Revision: 8d03a972ce8e92815ffe3d5d86aa027605ed92e2 URL: https://github.com/llvm/llvm-project/commit/8d03a972ce8e92815ffe3d5d86aa027605ed92e2 DIFF: https://github.com/llvm/llvm-project/commit/8d03a972ce8e92815ffe3d5d86aa027605ed92e2.diff

[clang] ab870f3 - Revert "PR47805: Use a single object for a function parameter in the caller and"

Author: Richard Smith Date: 2020-10-13T15:59:00-07:00 New Revision: ab870f3030fc288cd101905d730ef3425da5ac16 URL: https://github.com/llvm/llvm-project/commit/ab870f3030fc288cd101905d730ef3425da5ac16 DIFF: https://github.com/llvm/llvm-project/commit/ab870f3030fc288cd101905d730ef3425da5ac16.diff

[PATCH] D89348: clang/Basic: Replace ContentCache::getBuffer with Optional semantics

dexonsmith updated this revision to Diff 297987. dexonsmith added a comment. Fixed a bug where `Invalid` was only set when it was `true`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89348/new/ https://reviews.llvm.org/D89348 Files: clang-tools-extra/clang-tidy/ClangTidyDiagnosticCo

[PATCH] D89348: clang/Basic: Replace ContentCache::getBuffer with Optional semantics

dexonsmith updated this revision to Diff 297990. dexonsmith added a comment. Fix a few more places where `Invalid` needs to be set to `false`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89348/new/ https://reviews.llvm.org/D89348 Files: clang-tools-extra/clang-tidy/ClangTidyDiagnos

[PATCH] D87279: [clang] Fix handling of physical registers in inline assembly operands.

nickdesaulniers added a comment. https://bugs.llvm.org/show_bug.cgi?id=47837 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87279/new/ https://reviews.llvm.org/D87279 ___ cfe-commits mailing list cfe-comm

[PATCH] D78658: [clang][Frontend] Add missing error handling

dblaikie accepted this revision. dblaikie added inline comments. Comment at: clang/lib/Frontend/ASTUnit.cpp:1158-1160 + if (!Clang->hasTarget()) { return true; + } You can remove these braces now - since it's a superfluous change here now there's no need

[PATCH] D86671: [clang-tidy] Add new case type to check variables with Hungarian notation

dougpuob added a comment. Hi @aaron.ballman and @njames93, I addressed your code review suggestions and supported Hungarian Notation prefix for decl of enum and class(option) at latest patch. Unfortunately, I encountered a problem that new patch failed on remote BuildBot, it showed the "No such

[clang] 498c7fa - [SemaObjC] Fix a crash on an invalid ternary with ARC pointers

Author: Erik Pilkington Date: 2020-10-13T21:20:20-04:00 New Revision: 498c7fa48a02a5e0c12bbb90ef8c0d8468affcdd URL: https://github.com/llvm/llvm-project/commit/498c7fa48a02a5e0c12bbb90ef8c0d8468affcdd DIFF: https://github.com/llvm/llvm-project/commit/498c7fa48a02a5e0c12bbb90ef8c0d8468affcdd.dif

[PATCH] D34654: Allow passing a regex for headers to exclude from clang-tidy

srj added a comment. This is marked as "needs revision", but it's not clear to me what the requested changes actually are. The proposed fix here may not be perfect, but AFAICT, there is simply no good way to exclude headers that are beyond your control at present. Repository: rL LLVM CHANG

[PATCH] D78699: [X86] Passing union type through register.

LiuChen3 added a comment. In D78699#2328541 , @dyung wrote: > Hi, the test you added seems to pass both before and after your change, is > this intended? Oh, yes. I made mistake on my RUN line. So these tests are not really checked. Thanks for your help

[clang] 8f8b9f2 - PR47805: Use a single object for a function parameter in the caller and

Author: Richard Smith Date: 2020-10-13T18:50:46-07:00 New Revision: 8f8b9f2cca0b73314342c721186ae9c860ca273c URL: https://github.com/llvm/llvm-project/commit/8f8b9f2cca0b73314342c721186ae9c860ca273c DIFF: https://github.com/llvm/llvm-project/commit/8f8b9f2cca0b73314342c721186ae9c860ca273c.diff

[PATCH] D89184: Support complex target features combinations

LiuChen3 added a comment. > D89105 appears to use only `"avx512vl , > avx512vnni | avxvnni"`. > Does it mean `(avx512vl , avx512vnni) | avxvnni` or `avx512vl , (avx512vnni | > avxvnni)` ? Yes. "avx512vl , avx512vnni | avxvnni" means (avx512vl , avx512vnni) | a

[PATCH] D89360: Treat constant contexts as being in the default rounding mode.

rsmith created this revision. rsmith added reviewers: rjmccall, mibintc, sepavloff. Herald added a project: clang. rsmith requested review of this revision. This addresses a regression where pretty much all C++ compilations using -frounding-math now fail, due to rounding being performed in constex

[PATCH] D87528: Enable '#pragma STDC FENV_ACCESS' in frontend cf. D69272 - Work in Progress

rsmith added a comment. FYI: I posted a patch to implement the "manifestly constant evaluated" rule as https://reviews.llvm.org/D89360. It looks like the stricter rule doesn't work in practice; far too much C++ code uses `-frounding-math` and expects to be able to still include things like stan

[clang] 69f7c00 - Revert "PR47805: Use a single object for a function parameter in the caller and"

Author: Richard Smith Date: 2020-10-13T19:32:03-07:00 New Revision: 69f7c006ff72167231b569a0fd2d7955ca597ffa URL: https://github.com/llvm/llvm-project/commit/69f7c006ff72167231b569a0fd2d7955ca597ffa DIFF: https://github.com/llvm/llvm-project/commit/69f7c006ff72167231b569a0fd2d7955ca597ffa.diff

[clang] b72813f - [CMake][Fuchsia] Start building arm64 Darwin runtimes

Author: Petr Hosek Date: 2020-10-13T19:50:45-07:00 New Revision: b72813f4d6a0a0f1dc79afc1ca7e0e4d55ad6a72 URL: https://github.com/llvm/llvm-project/commit/b72813f4d6a0a0f1dc79afc1ca7e0e4d55ad6a72 DIFF: https://github.com/llvm/llvm-project/commit/b72813f4d6a0a0f1dc79afc1ca7e0e4d55ad6a72.diff LO

LLVM buildbot will be updated and restarted soon

Hello everyone, LLVM buildmaster will be updated and restarted in the nearest hour. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D89362: [MS] Apply `inreg` to AArch64 sret parms on instance methods

rnk created this revision. rnk added reviewers: mgrang, efriedma. Herald added subscribers: danielkiss, kristof.beyls. Herald added a project: clang. rnk requested review of this revision. The documentation rules indicate that instance methods should return large, trivially copyable aggregates via

[PATCH] D89360: Treat constant contexts as being in the default rounding mode.

rjmccall added a comment. Okay. I can accept the fairly unfortunate thing with `const int` initializers. When is `IsConstantContext` set in C, just static initializers? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89360/new/ https://reviews.llv

[PATCH] D89025: [RISCV] Add -mtune support

kito-cheng added a comment. > If possible that would good, since -mcpu is deprecated (for e.g. x86_64) or > unsupported in GCC (for e.g. RISC-V). So doing that would further align Clang > with GCC. But I wonder if this might be too problematic, in terms of > compatibility. I am also working on

[clang] bd05afc - [X86][NFC] Fix RUN line bug in the testcase

Author: Liu, Chen3 Date: 2020-10-14T12:40:34+08:00 New Revision: bd05afcb3f4093fe76dd64fe2920bfd894ec URL: https://github.com/llvm/llvm-project/commit/bd05afcb3f4093fe76dd64fe2920bfd894ec DIFF: https://github.com/llvm/llvm-project/commit/bd05afcb3f4093fe76dd64fe2920bfd894ec.diff LO

[PATCH] D89361: [X86][NFC] Fix RUN line bug in the testcase

This revision was automatically updated to reflect the committed changes. Closed by commit rGbd05afcb3f40: [X86][NFC] Fix RUN line bug in the testcase (authored by LiuChen3). Herald added a project: clang. Herald added a subscriber: cfe-commits. Changed prior to commit: https://reviews.llvm.org

[PATCH] D89361: [X86][NFC] Fix RUN line bug in the testcase

LiuChen3 added a comment. Thanks for all of your help! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89361/new/ https://reviews.llvm.org/D89361 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D89362: [MS] Apply `inreg` to AArch64 sret parms on instance methods

efriedma added inline comments. Comment at: clang/lib/CodeGen/MicrosoftCXXABI.cpp:1077 -static bool hasMicrosoftABIRestrictions(const CXXRecordDecl *RD) { +static bool isCXX14Aggregate(const CXXRecordDecl *RD) { // For AArch64, we use the C++14 definition of an aggregate, so

[PATCH] D89276: Support ObjC in IncludeInserter

compositeprimes updated this revision to Diff 298026. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89276/new/ https://reviews.llvm.org/D89276 Files: ___

[PATCH] D89276: Support ObjC in IncludeInserter

compositeprimes updated this revision to Diff 298027. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89276/new/ https://reviews.llvm.org/D89276 Files: clang-tools-extra/unittests/clang-tidy/IncludeInserterTest.cpp Index: clang-tools-extra/unittes

[PATCH] D89276: Support ObjC in IncludeInserter

compositeprimes updated this revision to Diff 298029. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89276/new/ https://reviews.llvm.org/D89276 Files: clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp clang-tools-extra/clang-tidy/utils/Include

[PATCH] D89184: Support complex target features combinations

LiuChen3 added inline comments. Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:2389 } -if (!hasRequiredFeatures(ReqFeatures, CGM, FD, MissingFeature)) +if (!llvm::all_of(ReqFeatures, [&](StringRef Feature) { + if (!CallerFeatureMap.lookup(Feature)) {

Re: [Lldb-commits] Upcoming upgrade of LLVM buildbot

They do on staging. http://lab.llvm.org:8014/#/builders/sanitizer-windows http://lab.llvm.org:8014/#/builders/clang-x64-windows-msvc Galina, when do you plan to push this to the primary server? If it's a few days, I'd rather keep bots on staging to have colors. On Tue, 13 Oct 2020 at 11:11, Rei

[PATCH] D89366: [WebAssembly] v128.load{8, 16, 32, 64}_lane instructions

tlively created this revision. tlively added reviewers: aheejin, dschuff. Herald added subscribers: llvm-commits, cfe-commits, ecnelises, sunfish, hiraditya, jgravelle-google, sbc100. Herald added projects: clang, LLVM. tlively requested review of this revision. Prototype the newly proposed load_

[clang] 625fa47 - Revert "[clang] Improve handling of physical registers in inline assembly operands."

Author: Jonas Paulsson Date: 2020-10-14T08:42:51+02:00 New Revision: 625fa47617022b2d1ed7b940f9621162874175ee URL: https://github.com/llvm/llvm-project/commit/625fa47617022b2d1ed7b940f9621162874175ee DIFF: https://github.com/llvm/llvm-project/commit/625fa47617022b2d1ed7b940f9621162874175ee.diff

[PATCH] D87279: [clang] Fix handling of physical registers in inline assembly operands.

jonpa reopened this revision. jonpa added a comment. This revision is now accepted and ready to land. Patch temporarily reverted. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87279/new/ https://reviews.llvm.org/D87279

<    1   2