This patch forms part of the ARMv8-M Baseline/Mainline support, allowing the use of wide branch and CBZ instructions in ARMv8-M Baseline.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
One minor comment on the logic here (though I don't think it would ever make a difference in practice):
lib/Target/ARM/ARMSubtarget.cpp | ||
---|---|---|
237 | Isn't the logic here really "!isThumb() || hasV8MBaselineOps()"? The Feature check part of isThumb1Only seems to be redundant since any T2-capable processor will have 8M baseline. Or am I misunderstanding? |
Comment Actions
Remove redundancy in condition check !isThumb1Only() && hasV8MBaselineOps(), all Thumb-2 targets will have hasV8MBaselineOps() set, so the first condition can simply be !isThumb().
Isn't the logic here really "!isThumb() || hasV8MBaselineOps()"? The Feature check part of isThumb1Only seems to be redundant since any T2-capable processor will have 8M baseline.
Or am I misunderstanding?