[PATCH] D152433: [ARM,AArch64] Add a full set of -mtp= options.
Nick Desaulniers via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 9 13:43:15 PDT 2023
nickdesaulniers added inline comments.
================
Comment at: llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp:4941-4943
if (LoadImmOpc == ARM::MRC || LoadImmOpc == ARM::t2MRC) {
- assert(Subtarget.isReadTPHard() &&
+ assert(!Subtarget.isReadTPSoft() &&
"TLS stack protector requires hardware TLS register");
----------------
>From the buildbot presubmit failure:
```
/var/lib/buildkite-agent/builds/llvm-project/llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp:4942:23: error: no member named 'isReadTPSoft' in 'llvm::ARMSubtarget'
assert(!Subtarget.isReadTPSoft() &&
~~~~~~~~~ ^
/usr/include/assert.h:93:27: note: expanded from macro 'assert'
(static_cast <bool> (expr) \
^~~~
```
PTAL
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152433/new/
https://reviews.llvm.org/D152433
More information about the llvm-commits
mailing list