1423287 - Simplify NtCurrentTeb handling

Status: RESOLVED FIXED

(Core :: General, enhancement)

Description

[(Away)]

Let's simplify the ifdef/asm mess, for three reasons:

1. There's no need for 64-bit-specific structs. PNT_TIB always "just works" in the current process. The arch-specific variants are provided for the sake of debuggers meddling with outside processes.

2. It seems GNUC doesn't need handholding anymore, per https://treeherder.mozilla.org/#/jobs?repo=try&revision=5807e726bd96

3. (My actual motivation) My revision of clang-cl trunk chokes on the selector syntax in the inline asm. It appears to be fixed on trunk tip, but I've got too many things in the air to update right now.

Comment 1

[(Away)]

Attachment: teb

njn, you seem like the most appropriate reviewer for this set of files, although I suspect you haven't used TEBs much. Maybe you can rubber-stamp if you trust my comment 0, otherwise feel free to reassign.

Comment 2

[(Away)]

(For some background, NtCurrentTeb() boils down to `__readfsdword(0x18)` on Win32, which has the same effect as the inline asm, with the added benefit that the function can do a similar thing with __readgsqword() on Win64, where inline asm is forbidden)

Comment 3

[Nicholas Nethercote [inactive]]

Comment on attachment 8934634 [details] [diff] [review]
teb

Review of attachment 8934634 [details] [diff] [review]:
-----------------------------------------------------------------

rs=me. I like how much duplicated code this patch removes!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b4d985311243
Simplify NtCurrentTeb() handling. r=njn

Comment 5

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/b4d985311243